Filtering improvements #182
Conversation
- Fix comparison operator to ensure we're comparing user ids - Do not disable the "any" option for second comparison choice - Fix clashing component ids
- Adds the option to group by prediction or by review. - Removes the "agree/disagree" grouping option as it's too ambiguous
- Add the options to sort by predictions or reviews - Remove the option to sort by agree/disagree because of ambiguity
Add "All Labels" option to the labels filter menu. Checking the "All Labels" option selects all labels. Unchecking removes all label filters. Labels can still be checked/unchecked individually. The "no label" filter remains its own option.
This adds "All Reviews" to the reviews filter menu to mirror the setup of the labels filter menu.
|
Recommendation: Show the "All Labels" as unchecked or in the ambiguous state if any of the labels are unchecked. I think. it is more likely you want to go back to showing all labels than showing no labels. |
|
I'm having trouble getting the filter by comparison section to do what I expect. I want to show where predictions differ from labels (or labels differ from predictions). I can't seem to pick labels on the left. When I pick predictions and differs from, picking labels on the right seems to go back to saying any. I'm not sure what is happening: filterby.mp4 |
For the comparison filters make sure that we confirm that the first option and operator have been set before trying to update any filtering.
bnmajor
force-pushed
the
filtering-improvements
branch
from
028d720 to
49fcd10
Compare
Updated!
This has been fixed now. |
groupByoptionsagree/disagreegrouping option as it's too ambiguoussortByoptionsagree/disagreebecause of ambiguityreviewsfilter menu to better match thelabelsfilter menu. This adds "All Reviews" to the reviews filter menu to mirror the setup of the labels filter menu.